Skip to content

Conversation

BenMorel
Copy link
Contributor

No description provided.

@BenMorel BenMorel requested a review from kocsismate as a code owner August 21, 2025 16:28
@BenMorel BenMorel changed the title Update array shape of gmp_div_qr() Update array shape of gmp_div_qr() stub Aug 21, 2025
@TimWolla
Copy link
Member

As can be seen in the failed CI, the generated arginfo is now out of sync with the stub file. This change also introduces a regression in the optimizer, because the stub generator doesn't understand the array shape syntax:

-	F1("gmp_div_qr", MAY_BE_ARRAY|MAY_BE_ARRAY_KEY_LONG|MAY_BE_ARRAY_OF_OBJECT),
+	F1("gmp_div_qr", MAY_BE_ARRAY|MAY_BE_ARRAY_KEY_LONG|MAY_BE_ARRAY_KEY_STRING|MAY_BE_ARRAY_OF_ANY|MAY_BE_ARRAY_OF_REF),

I'm therefore closing here. Before this can happen, support for array shapes must be added to the stub generator so that it generates proper optimizer information.

@TimWolla TimWolla closed this Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants